Skip to content

fix: align confirmation prompt behavior with (Y/n) convention#4350

Merged
ddoktorski merged 3 commits into
foundry-rs:masterfrom
mikhailUshakoff:fix/verify-prompt-default
May 29, 2026
Merged

fix: align confirmation prompt behavior with (Y/n) convention#4350
ddoktorski merged 3 commits into
foundry-rs:masterfrom
mikhailUshakoff:fix/verify-prompt-default

Conversation

@mikhailUshakoff

@mikhailUshakoff mikhailUshakoff commented May 8, 2026

Copy link
Copy Markdown
Contributor

Closes #4349

Introduced changes

  • This PR updates the sncast verify command prompt handling so that empty input is interpreted as "yes", aligning behavior with common CLI conventions and improving UX consistency.

Checklist

  • Linked relevant issue
  • Updated relevant documentation
  • Added relevant tests
  • Performed self-review of the code
  • Added changes to CHANGELOG.md

@mikhailUshakoff mikhailUshakoff requested a review from a team as a code owner May 8, 2026 20:51
Comment thread CHANGELOG.md
@ddoktorski

Copy link
Copy Markdown
Contributor

Hi @mikhailUshakoff
Would you be able to address the small feedback regarding the changelog, so we can get this merged? 🙏

@ddoktorski ddoktorski added this pull request to the merge queue May 29, 2026
Merged via the queue into foundry-rs:master with commit 1c75ba2 May 29, 2026
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Verify command fails on empty input despite (Y/n) default

3 participants